feat: exit with code 112 when api errors prevent cloud orchestrated runs (record mode, parallel run mode) #32635

cacieprins · 2025-10-02T20:39:32Z

Closes UX/CLI Improvement: Fatal Cy Cloud communication failure exits with a specific exit code #32485

Additional details

Adds 112 as a potential exit code when Cypress is run with POSIX complaint exit codes enabled (--posix-exit-codes). This exit code is used when --record or --parallel mode is unable to communicate with Cypress Cloud API due to network conditions.

Steps to test

How has the user experience changed?

PR Tasks

Have tests been added/updated?
Has a PR for user-facing changes been opened in cypress-documentation? docs: Adds documentation for the 112 exit code when posix-compliant exit codes are enabled. cypress-documentation#6294
[NA] Have API changes been updated in the type definitions?

Note

When recording/parallel runs hit Cloud network errors with --posix-exit-codes, exit with code 112; add network-specific error types and make Cloud API timeouts configurable.

Runner/CLI:
- Exit with code 112 for Cloud network failures in --record/--parallel when --posix-exit-codes is enabled; refactor startInMode and inline exitWithCode handling.
- Adjust run-mode exits to respect POSIX codes and cancellation; add extra debug logs.
Errors & Schema:
- Add CLOUD_CANNOT_PROCEED_IN_PARALLEL_NETWORK and CLOUD_CANNOT_PROCEED_IN_SERIAL_NETWORK to GraphQL schema and implement corresponding error templates and snapshots.
Cloud API:
- Introduce defaultTimeout() (honors CYPRESS_INTERNAL_API_TIMEOUT) and use it across requests instead of a fixed 60s timeout.
Stability:
- Tweak early-exit handling (EarlyExitTerminator.waitForEarlyExit signature).
Tests:
- Add/expand integration and system tests covering new exit code 112, network error paths, grouping/parallel behaviors, and timeout scenarios.
Changelog:
- Document new feature and related misc updates under 15.5.0.

^{Written by Cursor Bugbot for commit 65d7718. This will update automatically on new commits. Configure here.}

…re enabled

cypress · 2025-10-02T21:13:52Z

cypress Run #66432

Run Properties: Failed #66432 • 65d77181e3: Merge branch 'develop' into feat-api-failure-exit-112

Project	`cypress`
Branch Review	`feat-api-failure-exit-112`
Run status	`Failed #66432`
Run duration	`16m 49s`
Commit	`65d77181e3: Merge branch 'develop' into feat-api-failure-exit-112`
Committer	`Cacie Prins`
View all properties for this run ↗︎

Test results
Failures	`1`
Flaky	`1`
Pending	`705`
Skipped	`0`
Passing	`761`
View all changes introduced in this branch ↗︎

UI Coverage `0%`
Untested elements	`4`
Tested elements	`0`

Accessibility `100%`
Failed rules	`0 critical` `0 serious` `0 moderate` `0 minor`
Failed elements	`0`

Tests for review

cypress/e2e/studio/studio.cy.ts • 1 failed test • app-e2e

View Output

Test		Artifacts
Cypress Studio > updates an existing test with assertions		`Test Replay` `Screenshots`

cypress/e2e/cypress-origin-communicator.cy.ts • 1 flaky test • app-e2e

View Output

Test		Artifacts
Cypress In Cypress Origin Communicator > primary origin memory leak prevention > cleans up the primaryOriginCommunicator events when navigating away from the /specs to /runs		`Test Replay` `Screenshots`

AtofStryker · 2025-10-08T15:42:56Z

packages/server/lib/cypress.ts

-  return require('./errors').logException(err)
-  .then(() => {
-    debug('calling exit 1')
+  await require('./errors').logException(err)


any reason we can just import errors at the top of the file since we are starting to move away from CJS?

Because of how the integration tests are set up, this has to be dynamically required for now :(

AtofStryker · 2025-10-08T15:45:15Z

packages/server/lib/cypress.ts

+          break
+        }
+        case 'returnPkg': {
+          const pkg = await require('./modes/pkg')(options)


Can we just import pkg from '@packages/root' at the top of the file since its a bundled package now and just reference it here instead of ./modes/pkg? Would be much simpler

AtofStryker · 2025-10-08T15:45:36Z

packages/server/lib/cypress.ts

+          break
+        }
+        case 'info': {
+          await require('./modes/info')(options)


can we just hoist this import or await import it? If not I'll likely get to this in the launcher refactor

AtofStryker · 2025-10-08T15:47:42Z

system-tests/test/record_spec.js

        })
      })

+      it('errors and exits with 1 when posix exit codes are enabled', function () {


whats the reason the posix tests are added here and not in the posix PR? Is there some type of integration between these two I am missing?

oh I see its the differences between 112 vs 1 exit code

AtofStryker · 2025-10-08T15:49:51Z

packages/errors/test/__snapshots__/CLOUD_CANNOT_PROCEED_IN_PARALLEL_NETWORK.ansi

@@ -0,0 +1,8 @@
+[31mWe encountered an unexpected error communicating with our servers.[39m


this is so much easier to diff now

It really really is!

cli/CHANGELOG.md

jennifer-shehane · 2025-10-08T20:29:03Z

packages/server/lib/cypress.ts

+    }
+  }
+
+  debug('calling exit 1')


Can remove I believe

packages/server/lib/cypress.ts

jennifer-shehane · 2025-10-08T20:43:49Z

packages/server/lib/modes/record.ts

+import { getError, AllCypressErrorNames } from '@packages/errors'
+


Suggested change

import { getError, AllCypressErrorNames } from '@packages/errors'

import { getError } from '@packages/errors'

import type { AllCypressErrorNames } from '@packages/errors'

Addressed in 660ab1b

jennifer-shehane · 2025-10-08T20:45:43Z

packages/server/lib/modes/record.ts

            })
          default:
-            return throwErr('CLOUD_UNKNOWN_INVALID_REQUEST', {
+            throw throwErr('CLOUD_UNKNOWN_INVALID_REQUEST', {


I'm not understanding why this would change to throw either.

packages/server/test/integration/cypress_spec.js

cacieprins added 2 commits October 2, 2025 16:37

feat: exit with code 112 on fatal api errors when posix error codes a…

5ac2316

…re enabled

rm dead code

17c1002

cacieprins and others added 3 commits October 3, 2025 10:07

ts

79be6fe

system tests for 112 exit code

d401df8

Merge branch 'develop' into feat-api-failure-exit-112

d61e91d

jennifer-shehane assigned cacieprins Oct 3, 2025

cacieprins and others added 10 commits October 3, 2025 13:16

Merge branch 'develop' into feat-api-failure-exit-112

9affeaa

fix race condition w/ exit codes

3e891c9

only 112 on network errors, not http errors

2d5528b

have connection timeouts run a little bit faster in system tests

25724e9

Merge branch 'develop' into feat-api-failure-exit-112

c321884

new schema for network errors

12a8a1e

112 should only be on network errors, update integration test

e903179

changelog

b405587

Merge branch 'develop' into feat-api-failure-exit-112

afed8c5

changelog

eb8b28b

cacieprins marked this pull request as ready for review October 7, 2025 20:23

cacieprins marked this pull request as draft October 7, 2025 20:23

This comment was marked as outdated.

Sign in to view

cacieprins mentioned this pull request Oct 7, 2025

docs: Adds documentation for the 112 exit code when posix-compliant exit codes are enabled. cypress-io/cypress-documentation#6294

Open

cacieprins and others added 2 commits October 8, 2025 09:09

coerce boolean

ba640a1

Merge branch 'develop' into feat-api-failure-exit-112

edcf651

cacieprins marked this pull request as ready for review October 8, 2025 13:15

This comment was marked as outdated.

Sign in to view

cacieprins requested a review from AtofStryker October 8, 2025 13:52

AtofStryker reviewed Oct 8, 2025

View reviewed changes

jennifer-shehane self-requested a review October 8, 2025 20:22

jennifer-shehane requested changes Oct 9, 2025

View reviewed changes

code review

3f096f2

cacieprins and others added 3 commits October 10, 2025 14:28

review

660ab1b

rm extra debug

7c11f77

Merge branch 'develop' into feat-api-failure-exit-112

4152a21

cacieprins requested a review from jennifer-shehane October 10, 2025 18:31

Merge branch 'develop' into feat-api-failure-exit-112

65d7718

jennifer-shehane approved these changes Oct 14, 2025

View reviewed changes

cacieprins merged commit 86d9c84 into develop Oct 14, 2025
85 of 90 checks passed

cacieprins deleted the feat-api-failure-exit-112 branch October 14, 2025 15:35

		@@ -0,0 +1,8 @@
		[31mWe encountered an unexpected error communicating with our servers.[39m

		import { getError, AllCypressErrorNames } from '@packages/errors'

	import { getError, AllCypressErrorNames } from '@packages/errors'
	import { getError } from '@packages/errors'
	import type { AllCypressErrorNames } from '@packages/errors'

feat: exit with code 112 when api errors prevent cloud orchestrated runs (record mode, parallel run mode) #32635

feat: exit with code 112 when api errors prevent cloud orchestrated runs (record mode, parallel run mode) #32635

Uh oh!

Conversation

cacieprins commented Oct 2, 2025 • edited by cursor bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Additional details

Steps to test

How has the user experience changed?

PR Tasks

Uh oh!

cypress bot commented Oct 2, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Tests for review

Uh oh!

This comment was marked as outdated.

Uh oh!

This comment was marked as outdated.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

cacieprins commented Oct 2, 2025 •

edited by cursor bot

Loading

cypress bot commented Oct 2, 2025 •

edited

Loading